Skip to content

Conversation

dpvc
Copy link
Member

@dpvc dpvc commented Sep 17, 2025

This PR changes the clip-path specifications to use polygon() rather than xywh() in order to accommodate older browsers that don't implement xywh(), which is not as widely available as seems to be indicated in the listings at MDN (they don't indicate that some shapes aren't supported).

It also changes white-space: wrap to white-space: pre again to handle older browsers.

@dpvc dpvc requested a review from zorkow September 17, 2025 21:07
@dpvc dpvc added this to the v4.0.1 milestone Sep 17, 2025
Copy link

codecov bot commented Sep 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.72%. Comparing base (afaf1ea) to head (ea468ec).
⚠️ Report is 40 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop    #1356     +/-   ##
==========================================
  Coverage    86.72%   86.72%             
==========================================
  Files          337      337             
  Lines        84145    84156     +11     
  Branches      3140     4769   +1629     
==========================================
+ Hits         72971    72982     +11     
  Misses       11174    11174             

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@zorkow zorkow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. But I could not test due to lack of older browsers.

@dpvc dpvc merged commit 4d70d40 into develop Oct 10, 2025
3 checks passed
@dpvc dpvc deleted the update/clip-path branch October 10, 2025 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants